Skip to content

feat: Decal spawn, Playgunsound & FakeFire Methods / Firearm Module Propertys#842

Open
MS-crew wants to merge 32 commits into
ExMod-Team:devfrom
MS-crew:DecalSpawn
Open

feat: Decal spawn, Playgunsound & FakeFire Methods / Firearm Module Propertys#842
MS-crew wants to merge 32 commits into
ExMod-Team:devfrom
MS-crew:DecalSpawn

Conversation

@MS-crew

@MS-crew MS-crew commented Jun 2, 2026

Copy link
Copy Markdown

Description

Describe the changes
The spawn code was inspired by Dr.Dinspit`s code
https://github.com/icedchai/DarkAndGrittyGunplay/blob/main/DarkAndGrittyGunplay/Features/DecalRpcCache.cs

What is the current behavior? (You can also link to an open issue here)
No Decal Spawn

What is the new behavior? (if this is a feature change)
Yes Decal Spawn

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:
image


Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentations

Submission checklist

  • I have checked the project can be compiled
  • I have tested my changes and it worked as expected

Patches (if there are any changes related to Harmony patches)

  • I have checked no IL patching errors in the console

Other

  • Still requires more testing

@github-actions github-actions Bot added the API label Jun 2, 2026
@MS-crew MS-crew changed the title feat: Decal spawn Methods & Firearm ImpactEffectsModule property and helper method feat: Decal spawn Methods & Firearm ImpactEffectsModule property Jun 2, 2026
VALERA771
VALERA771 previously approved these changes Jun 3, 2026
Comment thread EXILED/Exiled.API/Features/Map.cs Outdated
Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>
@MS-crew MS-crew marked this pull request as draft June 3, 2026 19:48
@MS-crew MS-crew marked this pull request as ready for review June 3, 2026 19:53

@louis1706 louis1706 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obsolete the previous SpawnDecals that already exist

XingYeNotFish and others added 2 commits June 4, 2026 10:22
* refactor: CustomScpTermination

* for new cassie update

* fix: Remove breaking change

---------

Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>
@MS-crew

MS-crew commented Jun 4, 2026

Copy link
Copy Markdown
Author

Obsolete the previous SpawnDecals that already exist
@louis1706 done

@MS-crew MS-crew requested a review from louis1706 June 4, 2026 18:34
@MS-crew MS-crew marked this pull request as draft June 4, 2026 19:37
@MS-crew MS-crew marked this pull request as ready for review June 4, 2026 20:47
louis1706
louis1706 previously approved these changes Jun 7, 2026
@MS-crew MS-crew changed the title feat: Decal spawn Methods & Firearm ImpactEffectsModule property feat: Decal spawn, Playgunsound & FakeFire Methods / Firearm Module Propertys Jun 8, 2026
@MS-crew MS-crew changed the base branch from master to dev June 8, 2026 14:10
@MS-crew MS-crew dismissed stale reviews from louis1706 and VALERA771 June 8, 2026 14:10

The base branch was changed.

@MS-crew MS-crew requested review from VALERA771 and louis1706 June 8, 2026 14:21

@VALERA771 VALERA771 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

About names, I'd recommend something like SendFirearmRpc, but still I don't really think that we really need this methods cuz that's a basic Mirror

Comment thread EXILED/Exiled.API/Extensions/MirrorExtensions.cs Outdated
Comment thread EXILED/Exiled.API/Extensions/MirrorExtensions.cs Outdated
Comment thread EXILED/Exiled.API/Extensions/MirrorExtensions.cs Outdated
Comment thread EXILED/Exiled.API/Extensions/MirrorExtensions.cs Outdated
@MS-crew MS-crew requested a review from VALERA771 June 8, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants